Codecov Report

Attention: Patch coverage is 25.00000% with 18 lines in your changes missing coverage. Please review.

Project coverage is 70.50%. Comparing base (58098ed) to head (bd4fe6d).

@@            Coverage Diff             @@
##           master     #286      +/-   ##
==========================================
- Coverage   70.52%   70.50%   -0.02%     
==========================================
  Files         386      386              
  Lines       32299    32313      +14     
  Branches     4543     4546       +3     
==========================================
+ Hits        22779    22783       +4     
- Misses       8472     8482      +10     
  Partials     1048     1048              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

-- commits line 9 at r1:
Please rebase instead of merging. It makes the review much harder becaus it mixes the updates to main and the relevant updates. I will continue the review when it is rebased.

-- commits line 9 at r1: Please rebase instead of merging. It makes the review much harder becaus it mixes the updates to main and the relevant updates. I will continue the review when it is rebased.

@mudiagaobrikisil, are you waiting on the deuterocanon PR to go through to rebase? Or why has this stalled?

Addresses sillsdev/serval#477

OK, I rebased this. I thought we were planning on using step and total steps not a single double to communicate progress.

I think our strategy might ultimately differ from the one coded in this PR. Also, the issue itself might be pushed back/might be lower priority. For those reasons, I'm going to close this.